Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.x] (Doc+) Inference Pipeline ignores Mapping Analyzers (#112522) #112776

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

stefnestor
Copy link
Contributor

Backports the following commits to 8.x:

* (Doc+) Inference Pipeline ignores Mapping Analyzers

From internal Dev feedback (will cross-link after), this updates that inference processors within ingest pipelines run before mapping analyzers effectively ignoring them. So if users want analyzers to take effect, they would need to select the analyzer's ingest pipeline process equivalent and run it higher in flow than the inference processor.

---------

Co-authored-by: István Zoltán Szabó <[email protected]>
@stefnestor stefnestor added >docs General docs changes >enhancement auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Docs Meta label for docs team labels Sep 11, 2024
Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine merged commit b9662b5 into elastic:8.x Sep 11, 2024
6 checks passed
@stefnestor stefnestor deleted the backport/8.x/pr-112522 branch September 11, 2024 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >docs General docs changes >enhancement external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Docs Meta label for docs team v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants